-
Notifications
You must be signed in to change notification settings - Fork 200
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MAINT: パッケージ更新 #1466
MAINT: パッケージ更新 #1466
Conversation
"le": None, | ||
"ge": None, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
何故かOpenAPIスキーマーに
"le": 0,
"ge": 10,
が新しく追加されてしまったのでそれを回避するための修正です。
ただ存在しても動作自体に影響はないからこの修正はなくてもいいかも…
これもSkipJsonSchema[None]
の副作用のようです。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
うーーーーーんなるほどです!
この辺りもしちょっとよければ掃除してあげたいですね!!(別プルリクエストで)
ちょっと面倒かもなのですが、なんかいい感じにしていただけるとすごく助かります!!
この辺り詳しいの僕たちくらいなので。。。。。もしモチベあれば。。。
わかんないとこあったら何でも聞いていただければ (まあ僕もわからないのですが 😇 )
try: | ||
uvicorn.run(app, host=args.host, port=args.port) | ||
except KeyboardInterrupt: | ||
print("`KeyboardInterrupt` の検出によりエンジンを停止しました。") | ||
pass | ||
uvicorn.run(app, host=args.host, port=args.port) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
KeyboardInterruptが抑制されるようになったため削除。
https://www.uvicorn.org/release-notes/#0303-2024-07-20
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
starletteのバージョン固定のとこ以外lgtm だと思います!!
更新ありがとうございます!!
starletteとpip-licensesを修正しました。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
内容
依存パッケージの更新を行います。
関連 Issue
その他
^5.0.0
に変更しました。これによりメジャーバージョンが5に固定されます。